Skip to content

sotechsha でコンテンツ一覧を表示する #20

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Dec 12, 2016
Merged

Conversation

yasulab
Copy link
Member

@yasulab yasulab commented Dec 3, 2016

GET /sotechsha でindexアクションが反応し、ソーテック社の本に関する記事の一覧が表示される

cf. #18

@tomoko523
Copy link
Contributor

tomoko523 commented Dec 6, 2016

したこと

  • リダイレクト先のパーマリンクの設定をsotechshaの記事と区別がつくようにする
     ex. /gazou →/sotechsha-gazou (#は使用できませんでした)
  • blog_overview_page と同様に sotechsha_overview_page をgenarateする
  • /sotechsha のrouteをsotechsha_overview_pageに修正する
  • sotechsha_overview_pageの記事の並び順を1〜6,gazouにする
  • ↑テストを書く

しなかったこと

  • blog_overview_pageとsotechsha_overview_pageで重複する部分をhelper化する
    他の作業優先のため

@tomoko523 tomoko523 changed the title 🚧 [WIP] /sotechsha でコンテンツ一覧を表示する sotechsha でコンテンツ一覧を表示する Dec 8, 2016
@yasulab
Copy link
Member Author

yasulab commented Dec 12, 2016

@tomoko523 テストも書いてあって、いい感じです! 👌

今回はCoderDojo Advent Calendarに合わせてブログ記事を書きたいので、とりあえずはこれで一旦マージします🔧

お疲れ様でした〜 ♨️ 😆

@yasulab yasulab merged commit 6a0b276 into master Dec 12, 2016
@yasulab yasulab deleted the add-index-page branch December 17, 2016 03:36
@yasulab yasulab assigned tomoko523 and unassigned tomoko523 Jan 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants